Skip to content

Conversation

samrose
Copy link
Collaborator

@samrose samrose commented Nov 5, 2024

What kind of change does this PR introduce?

Minimum basis for supporting x86_64-darwin build

@samrose samrose marked this pull request as ready for review November 7, 2024 00:56
@samrose samrose requested a review from a team as a code owner November 7, 2024 00:56
Copy link
Contributor

@darora darora left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to build the artifacts as part of the CI builds? Having support for being able to do it sounds good though

@samrose
Copy link
Collaborator Author

samrose commented Nov 7, 2024

Do we need to build the artifacts as part of the CI builds? Having support for being able to do it sounds good though

It does help us to run these builds/checks. Usually a true full build only happens when something has changed in the source, otherwise it is passed over. Then, when some source is changed, the build for that part will happen, the results will be cached into our binary cache, and so then 90% of the rest of the future ci runs will be able to deterministically refer to and use the cached version. In the case of this build, adding a new system did trigger a full build, but that is also what we needed to test, and achieve.

So it is typical practice in nix based systems to run both builds and checks like this in ci. It should actually trigger a full build only in the small minority of cases. And in those cases, you almost always want to test the build, and cache it if it succeeds.

@samrose samrose merged commit 9edaf28 into develop Nov 7, 2024
13 checks passed
@samrose samrose deleted the x86-darwin branch November 7, 2024 01:38
damonrand pushed a commit to cepro/postgres that referenced this pull request Jun 15, 2025
* chore: adding support for x86_64-darwin

* fix: handle logic for various darwin platforms

* fix: != &&,  == ||

* fix: support system here too

* feature: support added for x86_64-darwin

* fix: increase timeout so that aws access does not expire

* fix: duration was too high

* chore: see if 2 hours will work

* fix: try more than 2 less than 3 hrs

* chore: try a bit lower

* chore: bump down lower

* chore: set to the apparent max

* chore: limit nix build to 15 on x86_64-darwin

---------

Co-authored-by: Sam Rose <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants